New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change the hierarchy issue messages #3016
Conversation
because they are difficult to translate with the verb variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Changes like this to support i18n better are good to have.
If you can toss a brief changelog entry into this PR, we'll get it merged in.
If anything is unclear about the process there, please ask.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know that these issues are not a result of your changes, but they should still be fixed regardless. Thank you for contributing.
redbot/cogs/admin/admin.py
Outdated
USER_HIERARCHY_ISSUE = _( | ||
"I tried to {verb} {role.name} to {member.display_name} but that role" | ||
HIERARCHY_ISSUE_REMOVE = _( | ||
"I tried to remove {role.name} to {member.display_name} but that role" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This phrase should be "I tried to remove {role.name} from ..."
redbot/cogs/admin/admin.py
Outdated
HIERARCHY_ISSUE_REMOVE = _( | ||
"I tried to remove {role.name} to {member.display_name} but that role" | ||
" is higher than my highest role in the Discord hierarchy so I was" | ||
" unable to successfully add it. Please give me a higher role and " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This phrase should be "unable to successfully remove it."
redbot/cogs/admin/admin.py
Outdated
) | ||
|
||
USER_HIERARCHY_ISSUE_REMOVE = _( | ||
"I tried to remove {role.name} to {member.display_name} but that role" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as above, "from" and "remove".
I think I did everything right now :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks solid, thanks!
Type
Description of the changes
Add custom messages for adding and removing roles. This removed the verb variable which makes it difficult to translate the messages.